Skip to content

Conversation

@anangl
Copy link
Contributor

@anangl anangl commented May 27, 2025

This is a follow-up to #21430. See individual commit messages.

anangl added 3 commits May 27, 2025 13:44
Set by default the `dir` field in the `nrfx_qspi2_xfer_t` structure
to `NRFX_QSPI2_XFER_DIR_TX` so that for transfers with no data the TX
direction is used. Leaving this field initialized to 0, what means
`NRFX_QSPI2_XFER_DIR_TXRX`, may cause problems with sQSPI v0.1.0 -
an extra pulse on the CS line may appear for SCK frequencies > 8MHz
and this can disturb further communication.

Signed-off-by: Andrzej Głąbek <[email protected]>
Add enabling and disabling of RAM for the sQSPI soft peripheral,
as described in the nRF54L15 porting guide.

Signed-off-by: Andrzej Głąbek <[email protected]>
This should be done only in a secure configuration.

Signed-off-by: Andrzej Głąbek <[email protected]>
@anangl anangl requested review from a team as code owners May 27, 2025 12:42
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label May 27, 2025
@anangl anangl requested a review from gbakke May 27, 2025 12:43
@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 27, 2025

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 1

Inputs:

Sources:

sdk-nrf: PR head: ff71b2c94b7caed03db1c8482b8b87dafb8f3366

more details

sdk-nrf:

PR head: ff71b2c94b7caed03db1c8482b8b87dafb8f3366
merge base: d0189a6959cded3ca35737a37359d2fde26b0fef
target head (main): 8f573d7995915a59aec46608d9085b8af169a2cb
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (1)
drivers
│  ├── mspi
│  │  │ mspi_sqspi.c

Outputs:

Toolchain

Version: 4aa3467a6d
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:4aa3467a6d_e85602c25f

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 58
  • ✅ Integration tests
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-boot
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_positioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread-main
    • test-low-level
    • test-sdk-audio
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

@carlescufi
Copy link
Contributor

@masz-nordic please take a look

@carlescufi carlescufi merged commit d7d3220 into nrfconnect:main May 29, 2025
14 checks passed
@anangl anangl deleted the fix_mspi_sqspi_no_data_transfers branch June 4, 2025 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants